alpine url bug fix and AGPL License version issue#3744
alpine url bug fix and AGPL License version issue#3744404-geek wants to merge 7 commits intoaboutcode-org:developfrom
Conversation
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
|
@404-geek Thanks... can you check the test failures? Also in the future it is usually best to avoid mixing two unrelated changes in the same PR... here the license detection issue is not related to anything about alpine VCS URL handling, AFAIK ;) |
…ression in pdfminer nexB#3740 Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Sure @pombredanne will take care of not keeping 2 issues in same PR in the future I have resolved all the test issues. Can you please check once |
AyanSinhaMahapatra
left a comment
There was a problem hiding this comment.
Thanks++ @404-geek , we are mostly there.
| @@ -1,5 +1,5 @@ | |||
| --- | |||
| license_expression: agpl-3.0-plus OR commercial-license | |||
There was a problem hiding this comment.
While the change is correct note that the rule filenames reflect the license-expression of the rule and a change in the license-expression needs to happen like this:
- Deprecate the current rule using the
is_deprecatedattribute - Reindex the licenses with
scancode-reindex-licenses(This will remove the deprecated rule from the index so you can add a new one, otherwise two rules with the same text cannot exist in the index) - Add a new rule for just
agpl-3.0-plusand add notes about the origin, that this was previously the src/agpl-3.0-plus_or_commercial-license_4.RULErule. Add the issue reference in the commit. (You can paste the rule content atetc/scripts/licenses/buildrules-template.txtand then runpython etc/scripts/licenses/buildrules.py etc/scripts/licenses/buildrules-template.txtin an activated virtualenv, and this will create the rule automatically, with the appropriate filename)
There was a problem hiding this comment.
Hi @AyanSinhaMahapatra I need some help on the buildrules-template.txt for generation.
|
@404-geek gentle ping, we need a small update here. |
|
@404-geek ping :) I would like to merge this soon enough! |
|
See #3740 (comment), will merge the commits thorugh #4093, closing. |
No description provided.